Skip to content
This repository was archived by the owner on Sep 2, 2024. It is now read-only.

Update hyperion to move just the aperture_scatterguard axes required#1544

Draft
DominicOram wants to merge 3 commits intomainfrom
1266_allow_individual_scatterguard_moves
Draft

Update hyperion to move just the aperture_scatterguard axes required#1544
DominicOram wants to merge 3 commits intomainfrom
1266_allow_individual_scatterguard_moves

Conversation

@DominicOram
Copy link
Collaborator

@DominicOram DominicOram commented Aug 20, 2024

Fixes DiamondLightSource/mx-bluesky#267

Link to dodal PR (if required):DiamondLightSource/dodal#758

To test:

  1. Do thing x
  2. Confirm thing y happens

AperturePosition.ROBOT_LOAD,
(ApertureInOut.OUT, None),
group="prepare_robot_load",
)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should do a set to (OUT, SMALL) small after this but will need to think about when to wait for it

def set_aperture():
yield from bps.abs_set(aperture_device, new_selected_aperture)
yield from bps.abs_set(
aperture_device, (ApertureInOut.IN, new_selected_aperture)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe should be out as we're going to do OAV after this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move scatterguard into correct position without moving aperture

1 participant